-
Notifications
You must be signed in to change notification settings - Fork 41
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
crane fix #1959
crane fix #1959
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM 👍
Many thanks @locallycompact for your help!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for your help @locallycompact
@locallycompact Many thanks for your PR ! Can you rebase it over main ? I would like to check that #1960 wasn't hiding other errors. Previously I tried a fix similar to yours and there was malloc errors that I could not explain. |
@locallycompact can you also sign the commits, as we can not merge the branch otherwise? |
90b26f8
to
284ffd9
Compare
By running 'nix flake update' command.
Rebased this now and also dropped the now invalid override |
Content
This PR includes a fix to the
nix
package which was broken after running anix flake update
command.Pre-submit checklist
Issue(s)
Closes #1928